gtkmenubutton: Popup menu/popover on GtkButton:clicked

This happens on button release, which is more convenient if the gesture
can be consumed by something else (eg. window dragging), and already behaves
correctly wrt cancelled gestures, broken grabs, etc.

This also allows us to unify pointer and keyboard behavior, popping up the
menu widget in a single place.

gesture: On cancel, remove touchpoint before checking the recognized state

Otherwise the touch being removed doesn't account in recognition.

button: Ensure the button is "left" when a gesture is cancelled

This prevents from stale widget states if the gesture happens to be
cancelled mid-press.

eventcontroller: Chain up on constructed

https://bugzilla.gnome.org/show_bug.cgi?id=745225

eventcontroller, widget: Don't crash if destroyed before the other

There are two scenarios. A widget sub-class owns a GtkEventController
and passes itself to it, or a controller owned by something else is
passed a widget.

In the second case, if the widget is destroyed before the controller,
we will have a crash when destructing the controller because we will
be accessing invalid memory. Adding a weak reference on the widget
addresses that problem.

This leads to a crash in the first case. When the widget is getting
destroyed, it will drop the reference to its own controller. The
controller will skip touching the widget because the weak reference
would have turned it to NULL. However, when the widget sub-class chains
up to GtkWidget it will try to free all the controllers in its list.
Unfortunately, all these controllers have already been destroyed. So
we need to guard against this too.

https://bugzilla.gnome.org/show_bug.cgi?id=745225

gtk_label_set_yalign(): Fix xalign/yalign typo in parameter name.

printing: Set printer state correctly

Consider NULL printer state message as empty too.
Printer state IPP_PRINTER_STOPPED can be considered as paused state.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

gtk_misc_set_alignment(): Improve the deprecation documenation.

To link to GtkWidget.halign/valign and to mention
GtkLabel.xalign/yalign.

gdkglcontext-win32: fix variable names in GDK_NOTE

These probably weren't caught because GDK_NOTE was disabled when the
change was tested.

https://bugzilla.gnome.org/show_bug.cgi?id=745400

gdkglcontext-win32: remove unused variables

https://bugzilla.gnome.org/show_bug.cgi?id=745400

Updated Czech translation

searchenginetracker: Unify fts/non-fts query

This makes sure we use location_uri for both types of query, and the fts
query has been made more similar to the one used by nautilus.

tracker: Turn on fts

We want to match what nautilus does, and the non-fts query
ignores the location, which yields somewhat broken UI in
the file chooser.

Formatting fixes

file chooser: Don't start search too eagerly

We were trying to start search when the user types anything,
but this is annoying more often than helpful, and interferes
with the location entry. So, stick with explicitly enabled
search (via the search button or Alt-S) for now.

Updated Slovak translation

Do not return NULL AtkStateSet

https://bugzilla.gnome.org/show_bug.cgi?id=741511

Updated translation for Afrikaans (af)

Updated Vietnamese translation

Signed-off-by: Trần Ngọc Quân <vnwildman@gmail.com>

wayland: Add debug spew

wayland: Formatting fixes

wayland: Apply maximized and fullscreen state

We were just throwing the request away if the app asks to
fullscreen or maximize a window before it has been mapped.
This is something the GdkWindow API explicitly supports,
so make it work by saving the state until the surface exists.

This fixes things under weston. There are bugs in mutter
that keep this from working correctly with gnome-shell.

https://bugzilla.gnome.org/show_bug.cgi?id=745303

wayland: Drop some pointless lines

No need for these parameter checks, really.

Updated Lithuanian translation

Fix a typo

Follow-up fix for commit 7cee6c5cc6f35d9ed453f943686c0b21e86b4a3e.

wayland: Don't use g_error() on connection lost

When the Wayland compositor vanishes, all applications connected will
receive a SIGPIPE as soon as they try to use wl_display_dispatch().

Do not use g_error() to terminate the applications when this occurs,
g_error() means an error in the application while here it's not truly
the case.

Use g_warning() and exit() instead.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
https://bugzilla.gnome.org/show_bug.cgi?id=745289

Updated Hungarian translation

Updated Hungarian translation

gdk: Use a better error message

Having "Assertion failed: success" on the console is not going to help
anyone.

Updated POTFILES.in and POTFILES.skip

Updated Polish translation

Updated Hebrew translation

GtkListBoxRow: add g_return_if_fail to grab_focus, don't crash

Although gtk_list_box_row_grab_focus() is not a public function
it can be easily called by gtk_widget_grab_focus() with a row argument
which has been removed from the list box and has box == NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=744879

Updated Greek translation

file chooser dialog: Prevent uneven button heights

The height of the text buttons depends on the font height,
whereas the search button has a fixed-size icon in it...
Prevent unevent heights by putting them all in a size group.

https://bugzilla.gnome.org/show_bug.cgi?id=745263

app chooser dialog: Prevent uneven button heights

The height of the text buttons depends on the font height,
whereas the search button has a fixed-size icon in it...
Prevent unevent heights by putting them all in a size group.

Add a configure check for new enough gettext

Check that xgettext understands glade files.

Stop building extract-strings

It is no longer needed.

Stop using extract-headers

Since 0.18.3, xgettext can extract strings from ui files.

Updated Slovak translation

Updated Greek translation

Updated Kazakh translation

printing: Always get details of Avahi printers via IPP

Request details of Avahi browsed printers even when
we've got their PPDs already. Some options are not
present in PPD.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

printing: Get covers for each printer individually

Since we combine Avahi browsed printers and
printers of local CUPS in one backend we need to
get covers for each printer separately.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

Updated Spanish translation

3.15.9

wayland: Don't allocate a full size SHM buffer when drawing using OpenGL

Before this patch, we'd always allocate a full size SHM buffer via
the wl_shm_pool, even though it would never be used. Instead allocate a
logical 1x1 cairo image surface.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Scale up wl_egl_window according to window scale

In order to support window scales for EGL windows, resize the
wl_egl_window to the window dimension multiplied with the window scale,
just as with SHM window buffers.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Properly support changing of surface scale

When the preferred surface scale changes, for example when entering a
wl_output with a higher scale than any previous entered output, recreate
the shm surface and redraw the window content with the new window scale.

Before this patch, the internal scale would be changed, but the shm
surface would not be recreated given the new scale, i.e. we'd attach a
buffer for a different scale than wl_surface.set_scale specified.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Don't ever try to set surface buffer scale when not supported

If the compositor is too old for handling surface buffer scales, never
tyr to set change it. This will effectively always leave it to its
initial state, i.e. 1.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

gdkcairo: Bail if surface is in error

Don't try to paint onto an error surface. This happens for example when
gdk_cairo_set_source_pixbuf() is called with a pixbuf that is too big
for Cairo to handle.

Spotted by Christian Boxdörfer

css: Fix _gtk_bitmask_subtract()

We were doing the wrong thing *and* writing uninitialized memory while
doing so. BAD.

Also added tests exposing these.

https://bugzilla.redhat.com/show_bug.cgi?id=1185585

Updated Galician translations

Updated POTFILES.in

Updated Spanish translation

Updated Slovenian translation

Updated Slovenian translation

Updated Hebrew properties translation

scrolledwindow: Only start fade animation on "visible" scrollbars

If a scrollbar is not shown (because of policies, or because it isn't
necessary), it doesn't make sense to start fade animations on its window
on captured motion events.

Updated Hebrew translation

Fix the previous commit

The C_() macro only takes string literals.
Use g_dpgettext2() instead

Make gtk-im-context-none work

This was added a few years ago, as a way to have _no_ im context
at all. But it didn't actually work. Make it work, and streamline
the handling of none by moving it all to gtkimmodule.c.

As part of this, add context to the translated names of all
im modules we ship.

icontheme: allow directories with mtime 0

In order to provide a constant mtime between OS build and deploy time,
while also maintaining a hardlink content-addressed model independent of
timestamps, ostree sets all mtimes to 0.

The icon cache code currently ignores directories with mtime 0, assuming
they don't exist.

Track directory existence in a more precise way.

https://bugzilla.gnome.org/show_bug.cgi?id=745052

getting-started: move Drawing section to bottom.

Updated Spanish translation

icontheme: use desired size instead of negative for DIR_UNTHEMED SVGs

When loading SVGs from ICON_THEME_DIR_UNTHEMED GtkIconInfos,
such as those created for a GLoadableIcon, the size of the pixbuf to
load is set as a product of icon_info->scale.
But a few lines above, icon_info->scale is set to -1 for
ICON_THEME_DIR_UNTHEMED GtkIconInfos, so we'll end up always passing a
negative size to the GdkPixbuf loader, which is interpreted as the
nominal size of the image file.
Instead load the SVG at the desired scaled size in that case.

This fixes blurry icon in the notification panel in gnome-shell.

https://bugzilla.gnome.org/show_bug.cgi?id=744991

icontheme: fill correct information when loading GResource-backed GIcons

When loading a GResource-backed GFileIcon into a GtkIconInfo we
currently fail to populate the is_resource private field.
Also, since is_svg is set by looking at the filename, and
g_file_get_path() returns NULL for a GResourceFile, is_svg was always
FALSE.

https://bugzilla.gnome.org/show_bug.cgi?id=744991

Updated POTFILES.in

Getting-started: port drawing.c to GtkApplication

range: Make steppers use smooth autoscrolling

This makes them work much nicer than the old timer-based
implementation.

range: Make zoom scrolling less sensitive

Use the new long press delay factor to make the zoom
scrolling less easy to trigger accidentally.

long press: Add a delay-factor property

Add a way to set a delay factor that can be used
to make the long press more or less sensitive.
Making this a factor instead of exposing the delay
itself preserves the value of the setting as an
overall 'slow down long press' setting.

zoom scrolling: Improve the previous fix

As Sebastian pointed out, just resetting the initial slider
position was an incomplete fix, because it does not cause the
delta to be recomputed, which is important in this scenario,
because you've likely travelled some distance over the slider
before the long press kicks in.
Instead, explicitly record both the slider position and the
delta.

zoom scrolling: avoid 'jumping back'

We record the starting position on button press, but only
start the zoom mode when the long press timeout kicks in.
Depending on circumstances, this can cause a noticable jump.

Avoid this by resetting the recorded starting position after
the long press timeout. Suggested by Sebastian Keller.

Updated Chinese (Taiwan) translation

Updated Chinese (Taiwan) translation

Silence a compiler warning

Updated Hungarian translation

icon browser: Set up dnd from the detail popup

Allow dragging the various icon sizes as images.

icon browser: Add some dnd support

Use a custom tree model and enable dragging from the icon view.
Currently, we just support dropping the icon name as text.

icon browser: Activate on single-click

We want to get away from double-clicks everywhere.

icon browser: stop selecting

We don't need the selection here.

gdk/quartz: Fix mixup in attributes usage creating window implementation

First, attributes can be NULL (which is always the case when calling
gdk_window_ensure_native) so do not unconditionally dereference it.

Then the window_type should be taken directly from the GdkWindow as
in other backends (such as the X11 one for example).

https://bugzilla.gnome.org/show_bug.cgi?id=744942

Updated Slovak translation

statusicon: missed setting one uID

statusicon: fix crash storing pointer in UINT

https://bugzilla.gnome.org/show_bug.cgi?id=696505

Updated Chinese (Taiwan) translation

Updated Chinese (Taiwan) translation

Updated Greek translation

3.15.8

menubutton: Fix possible button/popover state inconsistences

While a popover is hiding, the modal grab is already gone and the toggle
button is clickable again, but clicking again at that time will result in
gtk_widget_show() trying to show an already shown widget (although fading
out and hidden soon) and the toggle button activated.

So let the menubutton set the active status only if the menu/popover
widget wasn't already shown, and ensure this doesn't get triggered by
double/triple button press events.

inspector: Destroy popovers on unmap

As an implementation detail, the popover hooks the fade out animation
on ::hide. Destroying the popover right away here is not a problem, but
prevents the animation from actually running. ::unmap will be run after
the animation is finished, so destroy the popover there.

popover: Add show/hide transitions

These have the same visual effect and timing than the gnome-shell ones.
During the hide animation, the popover has been made to take focus
elsewhere, and refuse to take any pointer/keyboard input until the popover
is shown again.

This has been based on work from Timm Bäder.

https://bugzilla.gnome.org/show_bug.cgi?id=741405

scrolled window: Avoid a crash

gtk_scrolled_window_allocate_scrollbar was calling
gtk_scrolled_window_allocate_child just to get the relative
allocation, overlooking the fact that that function is only
safe to call if the scrolled window _has_ a child. Unfortunately,
gtk_scrolled_window_allocate_scrollbar will sometimes get called
when that is not the case. Since we are really only interested
in the relative allocation, just get that directly. This
fixes a segfault in the style-properties-nth-child reftest.

scrolled window: Drop dead code

The allocation is never used in this function, so don't bother
getting it.

printing: Use correct units for margins taken via IPP

cups_printer_get_hard_margins() returned margins in millimeters
for margins taken via IPP request instead of Units used before.

dnd: Set the parent DnD wayland surface on gtk_drag_set_icon_window()

This is a better place to call this, both lets us remove all other calls,
and additionally handles the gtk_drag_set_icon_widget() entry point.

Do not return on void functions

pathbar: use the same parameter as in the declaration

docs: Add extra symbol links to input handling documentation

Cross-reference to the API documentation more consistently.

https://bugzilla.gnome.org/show_bug.cgi?id=744054